refactor: full app rehaul — route-based architecture, component extraction, and UI unification#61
Merged
Merged
Conversation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A ground-up structural refactor of the Transcriptr app across 7 phases, followed by incremental feature and polish work. Net result: ~3,400 lines removed, clear route-based architecture, no more mobile/desktop component duplication, and a significantly smaller
TranscriptionStudio.What changed
Architecture (Phases 1–3)
/(upload),/transcribe/[id](processing + results),/studio/[id](full studio),/history,/about,/errors/[code]Headerwith responsive nav + mobile drawer replaces separateMobileHeader/MobileFooter/MainLayoutcomponentsComponent extraction (Phase 4)
TranscriptionStudiobig component into focused modules:AudioPlayer,FileDetails,TranscriptStatistics,ExportControls,EnhancedTranscript,KeyboardShortcutsModaluseAudioPlayerhook;TranscriptionStudiois now ~240 linesMobile/desktop deduplication (Phase 5)
MobileChangelog,MobileFeedbackForm,MobileFeedbackModals, and all mobile-only CSS filesDesign tokens (Phase 6)
V3AnnouncementModalandChangelogModalCleanup (Phase 7)
console.logcalls from client codeTranscriptionHistory.tsx,sequential-reveal-list.tsx, and otherdead files
Post-refactor features and fixes
Stats
Verified
bun x tsc --noEmit --ignoreDeprecations 6.0passesbun run buildpassesSummary by CodeRabbit
New Features
Bug Fixes
Chores